Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a bit of detail to tool-cache README.md #358

Closed
wants to merge 1 commit into from

Conversation

joshhale
Copy link

@joshmgross cleared up some fairly broad misunderstandings I had about the tool-cache. This is a suggested amendment to the README.md that would have set me on the right foot to begin with.

@joshhale
Copy link
Author

@joshmgross Should I cc a team to get some attention on this PR?

@joshmgross
Copy link
Member

cc: @hross and @bryanmacfarlane

@bryanmacfarlane
Copy link
Member

I have an existing doc that details how this works. I'm going to open source it and add it to this repo. So I think we should hold this PR for now. I'll create another PR and share

@bryanmacfarlane
Copy link
Member

#363

@joshhale
Copy link
Author

Sounds good to me. It will be nice to add a link from this README to the new doc once it's added.

@joshhale joshhale closed this Feb 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants